Fix stale path references in WARP.md#9336
Fix stale path references in WARP.md#9336anshul-garg27 wants to merge 2 commits intowarpdotdev:masterfrom
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @anshul-garg27 on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment |
|
I'm starting a first review of this pull request. You can follow along in the session on Warp. I approved this pull request and requested human review from: @vorporeal, @alokedesai, @zachbai. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This docs-only PR updates stale WARP.md path references to match the current repository layout and adjusts the workspace crate count wording.
Concerns
- None. The updated paths exist in the checkout, and no security-relevant code or configuration changes were introduced.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
Description
A few path references in WARP.md don't match the current workspace layout. The "Core Libraries" bullets list
warp_core/,editor/,ui/,ipc/, andgraphql/as if they live at the repo root, but each one is undercrates/. The "Workspace Structure" line still says "34+ member crates" —find crates -maxdepth 1 -mindepth 1 -type dreturns 63 today. And the Database / GraphQL sections point at paths that don't resolve:migrations/→crates/persistence/migrations/app/src/persistence/schema.rs→crates/persistence/src/schema.rsgraphql/api/schema.graphql→crates/warp_graphql_schema/api/schema.graphqlUpdated each reference to match what's on disk.
Testing
Docs-only change.